Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link rel preload supported as values #22728

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

tunetheweb
Copy link
Contributor

Summary

Add preload types as suggested in #22658 (review)

I'm not 100% sure when each of these as values support was added and not sure there's an easy way to check so have just copied the main preload values.

Currently the only difference is track which is unsupported by Firefox.

Test results and supporting details

https://wpt.fyi/results/preload?label=experimental&label=master&aligned

Related issues

Enhancement to #22658

@github-actions github-actions bot added the data:html 📄 Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML label Mar 28, 2024
@tunetheweb tunetheweb mentioned this pull request Mar 28, 2024
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I think it makes sense to copy the main preload values for these.
The idea here is to have data that differs for all future as= entries. This certainly is an improvement over marking the main preload feature as partial and mentioning all the unimplemented as= subfeatures in a note.

@Elchi3 Elchi3 merged commit eb2c7ef into mdn:main Apr 4, 2024
5 checks passed
@tunetheweb tunetheweb changed the title Link prel preload supported as values Link rel preload supported as values Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html 📄 Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants